-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OHRI-1994 Add TB Program menu and tabs shell on the patient chart #1695
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work with this. A couple of comments to consider before we get this in
@@ -2,3 +2,41 @@ export const tuberclosisMeta = { | |||
title: 'Tuberclosis', | |||
slotName: 'tuberclosis-slot', | |||
}; | |||
|
|||
export const TbDashboardDMeta = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export const TbDashboardDMeta = { | |
export const tbPatientChartMeta = { |
@@ -2,3 +2,41 @@ export const tuberclosisMeta = { | |||
title: 'Tuberclosis', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please delete this. I had added it as a place holder
packages/esm-tb-app/src/index.ts
Outdated
@@ -1,5 +1,12 @@ | |||
import { defineConfigSchema, getAsyncLifecycle, getSyncLifecycle } from '@openmrs/esm-framework'; | |||
import { createDashboardGroup, createDashboardLink } from '@openmrs/esm-patient-common-lib'; | |||
import { | |||
TbDashboardDMeta, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update this name after meta change
packages/esm-tb-app/src/routes.json
Outdated
"name": "ohri-tb", | ||
"slot": "patient-chart-dashboard-slot", | ||
"component": "patientChartTbDashboard", | ||
"order": 21 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this to 24 so that it appears HIV and COVID
const headerRecentTB = t('recentTuberculosis'); | ||
const headerPreviousCases = t('previousCases'); | ||
const headerVisit = t('visits'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add default incase a translation doesn't exist for example we don't have translations for this in Amaric
<Tab>{t('MdrTbEnrolment')}</Tab> | ||
<Tab>{t('PatientTracing')}</Tab> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment applies here. Change all occurrences of this within this PR
interface TbTreatmentFollowUpListProps { | ||
patientUuid: string; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reuse the interface defined in the commons. I think it's PatientChartProps
to avoid repetition
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ODORA0
Requirements
OHRI-123 My PR title
.Summary
Screenshots
Related Issue
Other